PR Summary

Medium Risk
Adds multiple new Slack API-backed tools and expands the Slack block parameter mapping/UI, which could affect existing workflow configurations and Slack permission requirements. Also slightly changes chat-execution detection logic, which could alter execution/streaming behavior for nonstandard callers.

Overview
Slack integration gains 4 new capabilities: slack_get_channel_info, slack_get_user_presence, slack_edit_canvas, and slack_create_channel_canvas, all registered in the tool registry and documented in docs/tools/slack.

Slack block UX and parameter mapping updated to expose these operations, add Slack user-selector inputs for ephemeral messages, user lookup, and presence checks (with advanced manual-ID fallbacks via canonicalParamId), and add canvas edit/create inputs. Includes a small download param fix to read the canonical fileName override.

Behavioral tweak: workflow execution logging/branching now treats “executing from chat” strictly as overrideTriggerType === 'chat' (removing inference from workflowInput).

^{Written by Cursor Bugbot for commit 1673fc0. Configure here.}

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Mar 5, 2026 6:11am

Greptile Summary

This PR adds four new Slack tools (slack_get_channel_info, slack_get_user_presence, slack_edit_canvas, slack_create_channel_canvas) and upgrades three existing param fields (ephemeralUser, userId, presenceUserId) from plain text inputs to user-selector dropdowns with canonical param IDs for backward compatibility. The overall structure follows established patterns in the codebase well.

Key concerns:

• rename operation will always fail at runtime — canvases.edit does not accept a rename operation; valid operations are insert_at_start, insert_at_end, insert_after, insert_before, replace, and delete. Canvas renaming requires a separate canvases.update API call. The title_content markdown-object format used in the rename branch is also not part of the canvases.edit change schema.
• Unrelated behavioral change — use-workflow-execution.ts drops the workflowInput-shape check from isExecutingFromChat. While the simplification is likely correct, it is entirely unrelated to Slack tools and should be a separate PR so the intent is clearly documented.
• Output naming inconsistency — create_channel_canvas returns canvas_id (snake_case) while every other output property in Slack tools uses camelCase.
• canvas.ts bug fix — a welcome addition: the existing canvas tool was missing a !data.ok check, silently swallowing API errors.

Confidence Score: 2/5

• Not safe to merge as-is — the Rename Canvas operation will consistently fail at the Slack API level for all users who select it.
• Three of the four new tools (get_channel_info, get_user_presence, create_channel_canvas) are solid. However, the edit_canvas tool includes a rename operation that targets the wrong Slack API endpoint and uses an incorrect payload shape, meaning it will always return an API error for end users. The unrelated isExecutingFromChat change, while likely benign, adds review surface without corresponding context or tests.
• apps/sim/tools/slack/edit_canvas.ts requires a fix to either remove the rename operation or route it to canvases.update.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant UI as Slack Block UI
    participant Block as slack.ts (config)
    participant Registry as tools/registry.ts
    participant Tool as Slack Tool
    participant API as Slack API

    UI->>Block: operation = get_channel_info
    Block->>Registry: resolve → slack_get_channel_info
    Registry->>Tool: slackGetChannelInfoTool
    Tool->>API: GET conversations.info?channel=C123
    API-->>Tool: { ok: true, channel: {...} }
    Tool-->>UI: channelInfo object

    UI->>Block: operation = get_user_presence
    Block->>Registry: resolve → slack_get_user_presence
    Registry->>Tool: slackGetUserPresenceTool
    Tool->>API: GET users.getPresence?user=U123
    API-->>Tool: { ok: true, presence: "active" }
    Tool-->>UI: presence (+ null detail fields for non-self)

    UI->>Block: operation = edit_canvas (insert_at_end)
    Block->>Registry: resolve → slack_edit_canvas
    Registry->>Tool: slackEditCanvasTool
    Tool->>API: POST canvases.edit { canvas_id, changes:[{operation}] }
    API-->>Tool: { ok: true }
    Tool-->>UI: "Successfully edited canvas"

    UI->>Block: operation = edit_canvas (rename) ❌
    Block->>Registry: resolve → slack_edit_canvas
    Registry->>Tool: slackEditCanvasTool
    Tool->>API: POST canvases.edit { changes:[{operation:"rename"}] }
    API-->>Tool: { ok: false, error: "invalid_arguments" }
    Tool-->>UI: Error thrown

    UI->>Block: operation = create_channel_canvas
    Block->>Registry: resolve → slack_create_channel_canvas
    Registry->>Tool: slackCreateChannelCanvasTool
    Tool->>API: POST conversations.canvases.create { channel_id }
    API-->>Tool: { ok: true, canvas_id: "F123" }
    Tool-->>UI: canvas_id

_{Last reviewed commit: 1673fc0}

@cursor review

@greptile

@greptile

@cursor review